Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub Actions 上で型検査する #39

Merged
merged 5 commits into from
Sep 24, 2021
Merged

GitHub Actions 上で型検査する #39

merged 5 commits into from
Sep 24, 2021

Conversation

mashabow
Copy link
Owner

CI では設定を忘れていたので。

が良さそうだったが、andoshin11/typescript-error-reporter-action#13 によると最近の TS では動かない様子。そこで、この issue で紹介されているフォークを使ってみる。

https://github.com/Gozala/typescript-error-reporter-action

return json.members!.reduce<Record<string, string>>(
return json.members.reduce<Record<string, string>>(
Copy link
Owner Author

@mashabow mashabow Sep 24, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

試しに変更してみたら PR の diff view に annotation が付いた。

GitHub_Actions_上で型検査する_by_mashabow_·_Pull_Request__39_·_mashabow_slack-rota

https://github.com/mashabow/slack-rota/pull/39/files/ca5aea5912b9d89a5996ff0f84036ec1bbde5149..8ef881df6224d78dd78069a9c8ddcd2c1d418fb6#diff-d7add22db7e289142aea1a04d5d5ea6e8f9d2c1e9b31d3189a29d42e5860c299

どこのトークンでエラーになっているのかが表示されないので、そこが若干微妙…

@mashabow mashabow marked this pull request as ready for review September 24, 2021 17:54
@mashabow mashabow merged commit 9ce5a78 into master Sep 24, 2021
@mashabow mashabow deleted the ts-github-actions branch September 24, 2021 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant